Skip to content

fix: [UIE-10244] - Connection Pools table page size#13448

Open
hana-akamai wants to merge 5 commits intolinode:developfrom
hana-akamai:UIE-10244-connection-pools-table-page-size
Open

fix: [UIE-10244] - Connection Pools table page size#13448
hana-akamai wants to merge 5 commits intolinode:developfrom
hana-akamai:UIE-10244-connection-pools-table-page-size

Conversation

@hana-akamai
Copy link
Contributor

@hana-akamai hana-akamai commented Feb 27, 2026

Description 📝

Remove Infinity option from the Database Connection Pools table page size dropdown

Changes 🔄

  • Remove Infinity option from the Database Connection Pools table page size dropdown
  • Add DEFAULT_PAGE_SIZES constant without Infinity to PaginationFooter constants

Scope 🚢

Upon production release, changes in this PR will be visible to:

  • All customers
  • Some customers (e.g. in Beta or Limited Availability)
  • No customers / Not applicable

Target release date 🗓️

Next release

Preview 📷

Before After
image image

How to test 🧪

Prerequisites

(How to setup test environment)

  • Use the legacy MSW and update the number of connection pools on line 435 of serverHandlers.ts to a bigger number like 50

Reproduction steps

(How to reproduce the issue, if applicable)

  • On develop or another branch, update the number of connection pools to >25
  • Click on the page size dropdown and there is an Infinity option

Verification steps

(How to verify changes)

  • Checkout this PR, update the number of connection pools to >25
  • Click on the page size dropdown and there should not be an Infinity option

@hana-akamai hana-akamai self-assigned this Feb 27, 2026
@hana-akamai hana-akamai added Bug Fixes for regressions or bugs DBaaS Relates to Database as a Service labels Feb 27, 2026
@hana-akamai hana-akamai marked this pull request as ready for review February 27, 2026 22:46
@hana-akamai hana-akamai requested a review from a team as a code owner February 27, 2026 22:46
@@ -1,3 +1,4 @@
export const MIN_PAGE_SIZE = 25;

export const DEFAULT_PAGE_SIZES = [MIN_PAGE_SIZE, 50, 75, 100];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be somewhere else? It's not obvious to me what the difference is between DEFAULT_PAGE_SIZES and PAGE_SIZES below

Copy link
Member

@bnussman-akamai bnussman-akamai Mar 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe PAGE_SIZES can be removed and we can just use DEFAULT_PAGE_SIZES only? From what I remember, we don't let people select Infinity/All anymore anyway

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point, moved DEFAULT_PAGE_SIZES to the Database constants file for now. I'm not sure if we removed the select Infinity/All - posted the question in team channel

Copy link
Contributor

@smans-akamai smans-akamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tested out the changes and it looks great! Thanks for addressing my feedback.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Fixes for regressions or bugs DBaaS Relates to Database as a Service

Projects

Status: Review

Development

Successfully merging this pull request may close these issues.

3 participants